Skip to content

Run all pipeline commands locally#7

Open
Staberinde wants to merge 11 commits intomainfrom
run-all-pipeline-commands-locally-take-2
Open

Run all pipeline commands locally#7
Staberinde wants to merge 11 commits intomainfrom
run-all-pipeline-commands-locally-take-2

Conversation

@Staberinde
Copy link
Copy Markdown
Contributor

@Staberinde Staberinde commented Jan 12, 2022

This allows the pipeline to be run locally with no prerequisites other than docker

dockerised is the target to use, and targets to run within docker should be specified within TARGETS argument e.g. make TARGET="collect collection dataset" dockerised

If you want to avoid a risk of contaminating the main collection, pass DEVELOPMENT=1 as an argument, and any outputs created will be created in a local_collection subfolder.

I've tried to make this as unobtrusive and self-contained a possible, but that's lead to some quite ugly implementation in places

* Build image for component with python deps and /pipeline
* Run makefile inside component
Command used:
`make TARGET="fetch-organisation collect collection dataset" DEVELOPMENT=1 DISABLE_DOCKER_PULL=1 docker-build mk-local-collection dockerised`
Changes:
* Always create harmonised and fixed directories before docker run
* Create transformed issue and dataset directories before docker run
* Work around weird issues with init inheritence not working
Outstanding refactors:
* Do we need all these directories
* Why isn't init inheritence working?
@Staberinde Staberinde changed the title Run all pipeline commands locally take 2 Run all pipeline commands locally Jan 14, 2022
@Staberinde Staberinde marked this pull request as ready for review January 14, 2022 13:23
@Staberinde Staberinde force-pushed the run-all-pipeline-commands-locally-take-2 branch from 09f5534 to e016714 Compare January 14, 2022 13:24
… run-all-pipeline-commands-locally-take-2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant